Skip to content

Conversation

@manoncarbonnel
Copy link

@manoncarbonnel manoncarbonnel commented Oct 24, 2025

Description

Adding special columns for Cloud GPUcategory of Flavors

Ticket Reference: #TAPC-5145 #TAPC-5217

Screenshots

image

@manoncarbonnel manoncarbonnel self-assigned this Oct 24, 2025
@manoncarbonnel manoncarbonnel changed the base branch from master to feat/add-instance-creation-page-TAPC-4445 October 24, 2025 08:54
@manoncarbonnel manoncarbonnel force-pushed the feat/gpu-flavors-TAPC-5145 branch 2 times, most recently from f1ee7c1 to 0fc19e2 Compare October 24, 2025 13:29
@github-actions github-actions bot added the has conflicts Has conflicts to resolve before merging label Oct 29, 2025
@manoncarbonnel manoncarbonnel force-pushed the feat/gpu-flavors-TAPC-5145 branch from 0fc19e2 to 5b7c2d2 Compare October 30, 2025 10:29
@github-actions github-actions bot removed the has conflicts Has conflicts to resolve before merging label Oct 30, 2025
@manoncarbonnel manoncarbonnel force-pushed the feat/gpu-flavors-TAPC-5145 branch 3 times, most recently from 4351e74 to 876c56a Compare October 30, 2025 13:52
@manoncarbonnel manoncarbonnel marked this pull request as ready for review October 30, 2025 13:53
@manoncarbonnel manoncarbonnel requested a review from a team as a code owner October 30, 2025 13:53
@manoncarbonnel manoncarbonnel force-pushed the feat/gpu-flavors-TAPC-5145 branch 2 times, most recently from 517a814 to 3e45c38 Compare October 31, 2025 09:30
@github-actions github-actions bot added the has conflicts Has conflicts to resolve before merging label Oct 31, 2025
Copy link
Contributor

@Tsiorifamonjena Tsiorifamonjena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What a big work 💪🏽👏🏽🎊

All good for me except the conflict

@manoncarbonnel manoncarbonnel force-pushed the feat/gpu-flavors-TAPC-5145 branch 3 times, most recently from 05a25f5 to 180fe31 Compare October 31, 2025 11:02
@github-actions github-actions bot removed the has conflicts Has conflicts to resolve before merging label Oct 31, 2025
Copy link
Contributor

@lolottetheclash lolottetheclash left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

well done, big step!!!💪

@manoncarbonnel manoncarbonnel force-pushed the feat/gpu-flavors-TAPC-5145 branch 4 times, most recently from 89689ef to 7080290 Compare October 31, 2025 13:50
ref: #TAPC-5145 #TAPC-5217

Signed-off-by: Manon Carbonnel <[email protected]>
Co-authored-by: CDS Translator Agent <[email protected]>
Signed-off-by: Manon Carbonnel <[email protected]>
@manoncarbonnel manoncarbonnel force-pushed the feat/gpu-flavors-TAPC-5145 branch from 7080290 to 63d3454 Compare October 31, 2025 13:57
Copy link
Contributor

@aTurmo aTurmo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really great :D
Just a little something that I find wierd ^^

Comment on lines +31 to +32
const list =
selectedCategory === 'Cloud GPU' ? mockedGpuFlavors : mockedFlavors;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Question (could be blocking): is it normal that there are mocked values here ?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes for now it is, but real data are coming soon, after modeling

@manoncarbonnel manoncarbonnel merged commit 6f2754b into feat/add-instance-creation-page-TAPC-4445 Oct 31, 2025
14 checks passed
@manoncarbonnel manoncarbonnel deleted the feat/gpu-flavors-TAPC-5145 branch October 31, 2025 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants